widget: don't check before setting to avoid costly type checks
authorStefan Kost <ensonic@users.sf.net>
Thu, 11 Feb 2010 08:38:44 +0000 (10:38 +0200)
committerStefan Kost <ensonic@users.sf.net>
Mon, 15 Feb 2010 08:14:50 +0000 (10:14 +0200)
There are no side effect of setting these flags and we don't fire g_object
notifies also. Thus set/unset right away without costly check of current state.

gtk/gtkwidget.c

index a18fc0c259049328ae94b53e000327589d58e30f..953440eb3aa328249677916f2d732263275fa12f 100644 (file)
@@ -5986,15 +5986,10 @@ gtk_widget_set_realized (GtkWidget *widget,
 {
   g_return_if_fail (GTK_IS_WIDGET (widget));
 
-  realized = realized != FALSE;
-
-  if (realized != gtk_widget_get_realized (widget))
-    {
-      if (realized)
-        GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
-      else
-        GTK_WIDGET_UNSET_FLAGS (widget, GTK_REALIZED);
-    }
+  if (realized)
+    GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
+  else
+    GTK_WIDGET_UNSET_FLAGS (widget, GTK_REALIZED);
 }
 
 /**
@@ -6033,15 +6028,10 @@ gtk_widget_set_mapped (GtkWidget *widget,
 {
   g_return_if_fail (GTK_IS_WIDGET (widget));
 
-  mapped = mapped != FALSE;
-
-  if (mapped != gtk_widget_get_mapped (widget))
-    {
-      if (mapped)
-        GTK_WIDGET_SET_FLAGS (widget, GTK_MAPPED);
-      else
-        GTK_WIDGET_UNSET_FLAGS (widget, GTK_MAPPED);
-    }
+  if (mapped)
+    GTK_WIDGET_SET_FLAGS (widget, GTK_MAPPED);
+  else
+    GTK_WIDGET_UNSET_FLAGS (widget, GTK_MAPPED);
 }
 
 /**